Skip to content

Conversation

@eiresendez
Copy link
Contributor

@eiresendez eiresendez commented Oct 29, 2025

Issue

When config('smart-extract.hitl_enabled') is true, the /api/1.0/tasks endpoint still applies the nonSystem() scope, so the HITL task “Manual Document Review” never reaches the /tasks UI. Once visible, its case_number remains null, so the task list renders “undefined”. Reproduce by enabling HITL, assigning the task to a user and loading /tasks with non_system=true.

Solution

  • Adjusted excludeNonVisibleTasks to keep filtering system tasks while explicitly whitelisting the “Manual Document Review” HITL task when the HITL feature flag is on.
  • Augmented the Task API resource to inject the parent request’s case_number for that HITL task so the UI receives a complete payload.
  • Simplified the Vue task list renderer now that the backend provides the correct case number.

How to Test

  1. Set smart-extract.hitl_enabled to true and confirm a “Manual Document Review” task is assigned to the test user.
  2. Call /api/1.0/tasks?include=process,processRequest,processRequest.user,user&non_system=true and verify the task appears with a populated case_number.
  3. Visit /tasks and confirm the row shows the HITL task with the correct case number, while other system tasks remain hidden.

Related Tickets & Packages

CarliPinell and others added 30 commits December 5, 2024 15:42
…Expressio

 ## Description:
Steps to Reproduce the Issue:
  Open a Request/Case:
Navigate to the ProcessMaker cases and open an existing request (case).
Click on the Reassignment Button:
Locate and click the "Reassignment" button to reassign the case to another user.
  View the Assignment User List:
Observe the list of users displayed in the reassignment dropdown.
  Current Behavior (Issue):
Instead of filtering users as per the regular expression, all users are displayed in the reassignment list.
  Expected Behavior:
The user list should be filtered based on the regular expression condition set in the process.
Only users that match the specified regular expression should appear in the list.
  Impact:
The client expects the user list to be filtered dynamically based on the regular expression condition applied in the process, but it is not working correctly.
  Potentials Fix:
Request the development team to investigate and ensure that the reassignment user list correctly applies the regular expression condition as configured in the process.
  Acceptance criteria:
When having a Rule Expression, the reassignment should only allow to reassign to Admin, Process Manager or the users listed in the Rule Expression list.

 ## Related Tickets & Packages
https://processmaker.atlassian.net/browse/FOUR-23734
nolanpro and others added 23 commits October 27, 2025 11:49
Fix error when dependent screen is missing
FOUR-25919: Processmaker is not registering the name of user performing a task on behalf of someone else
Textarea Control - Rich text does not work correctly in nested screens
FOUR-24085: Email Screen is not displaying its styles when it assigned to an Action By Email
observation/FOUR-21268 Mobile Version doesn't have a close button for the hamburger
FOUR-24683: [45464] Richtext field not rendering ul tag (regression)
FOUR-25910: Old processes do not update the progress bar status (Error and Canceled)
Bugfix/FOUR-21536: [42898] UI Doesn't Allow Scrolling in Request Details Panel
bugfix/FOUR-19358: [40847] Bug: Sorting Functionality Not Working for Script Executors in Admin Panel
Bugfix/FOUR-26739: Magic variable _request.name not working inside calc properties or visibility rules
[Regression] SAML admin users prompted for password when updating user details
Process Manager Granular Permissions
FOUR-23734 [44349] - Reassignment User List Not Filtering by Regular …
Server Version Disclosure On Error Page
Process Manager - Number of Users to Assign
Copy link
Contributor

@CarliPinell CarliPinell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works fine!

@eiresendez eiresendez changed the base branch from develop to epic/FOUR-26611 October 30, 2025 16:20
@processmaker-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@eiresendez
Copy link
Contributor Author

Closing in favor of: #8580 (clean)

@eiresendez eiresendez closed this Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.